Skip to content

Feat: use 1 thread for dbt duckdb targets#2222

Merged
treysp merged 2 commits intomainfrom
trey/dbt-duckdb-threads
Mar 6, 2024
Merged

Feat: use 1 thread for dbt duckdb targets#2222
treysp merged 2 commits intomainfrom
trey/dbt-duckdb-threads

Conversation

@treysp
Copy link
Contributor

@treysp treysp commented Mar 5, 2024

DuckDB doesn't support concurrent writes, so we automatically use 1 thread even if the dbt profile specifies >1

@treysp treysp requested a review from crericha March 5, 2024 23:41
Copy link
Contributor

@crericha crericha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a test?

@treysp treysp force-pushed the trey/dbt-duckdb-threads branch from 9dfbc17 to 4847f67 Compare March 6, 2024 00:06
@treysp treysp merged commit 35d430c into main Mar 6, 2024
@treysp treysp deleted the trey/dbt-duckdb-threads branch March 6, 2024 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants